Release 0.3.5 with corrected shared lifecycle guidance - #7
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
PR Summary by QodoRelease 0.3.5 with corrected public lifecycle guidance
AI Description
Diagram
High-Level Assessment
Files changed (6)
|
Code Review by Qodo
1. Public uploads lack automatic hooks
|
| Lifecycle hooks need CLI 0.15.6 or newer; a skills-only plugin package has no | ||
| automatic hooks. Use `integrations status --verbose` to inspect the last | ||
| Lifecycle hooks need CLI 0.15.6 or newer; older packages may omit | ||
| automatic hooks. Current Codex packages include them, including the public upload. Use `integrations status --verbose` to inspect the last |
There was a problem hiding this comment.
1. Public uploads lack automatic hooks 🔗 Cross-repo conflict ≡ Correctness
README.md and skills/tree-ring-memory/SKILL.md claim that current public Codex uploads include automatic hooks, while Tree_Ring_Memory's public-upload builder removes the hooks manifest entry and all /hooks/ files and TERMS.md still defines the separate OpenAI upload as skills-only. Users who install that artifact or consult the terms will receive contradictory lifecycle guidance and may expect automatic recall and checkpoints that still require separate CLI or project-hook configuration.
Agent Prompt
## Issue description
Correct the lifecycle guidance for the current Codex public-upload artifact. The README and skill currently say it includes automatic hooks, while the artifact remains skills-only and `TERMS.md` describes it that way.
## Issue Context
The canonical Tree_Ring_Memory builder removes the hook declaration and validates that no hook files enter the public ZIP, so automatic recall and checkpoints still require separate CLI or project-hook configuration. Either restore the skills-only warning in the README and skill, or coordinate and release the upstream package change before claiming public-upload hook support; then make the terms and all package descriptions consistently distinguish older packages from current Git and public-upload packages.
## Fix Focus Areas
- TERMS.md[8-10]
- README.md[202-207]
- skills/tree-ring-memory/SKILL.md[33-36]
ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools
Correct shared skill and README guidance: current Codex public uploads also include native lifecycle hooks; older packages may omit them. Keep the native Claude hooks and runtime contract unchanged.
Validation: Claude package validation passed; runtime remains 0.15.7.
High-level PR Summary
This release bumps the version to 0.3.5 and corrects documentation guidance about lifecycle hooks. The changes clarify that current Codex public uploads include native lifecycle hooks, while older skills-only packages may omit them. All modifications are documentation updates to reflect the current state of the package, with version number updates across configuration files and validation scripts to maintain consistency.
⏱️ Estimated Review Time: 5-15 minutes
💡 Review Order Suggestion
skills/tree-ring-memory/SKILL.mdREADME.md.claude-plugin/plugin.json.claude-plugin/marketplace.jsonSUBMISSION.mdscripts/validate.py